-
Notifications
You must be signed in to change notification settings - Fork 191
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: improve documentation #90
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per JSDoc docs
Updated to requested changes, however there is just a nit - the |
Shouldn't |
thats what i thought, but #90 (comment) |
5ab576b
to
36de77d
Compare
Could throw in an index.d.ts while you're at it. |
see #89 (comment) snek doesn't want typings in the library itself |
This PR adds some better documentation, corrects some invalid syntax in the JSDoc comments, and deprecates
RPCClient#sendJoinRequest
as the command has been removedthis PR only changes documentation
reference: this pr is apart of the chain of prs requested from #89 (comment)